Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ChipInfo struct #494

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Add ChipInfo struct #494

merged 1 commit into from
Feb 6, 2025

Conversation

pjanevskiTT
Copy link
Contributor

Issue

/

Description

Tearing apart the PR for blackhole topology into smaller ones.
This PR adds ChipInfo struct which holds all the information needed to construct the Chip on top of TTDevice.

List of the changes

  • Add ChipInfo struct
  • Read ChipInfo struct from TTDevice
  • Add test

Testing

Added test. Existing CI

API Changes

/

@pjanevskiTT pjanevskiTT requested a review from broskoTT January 29, 2025 13:18
@pjanevskiTT pjanevskiTT self-assigned this Jan 29, 2025
@pjanevskiTT pjanevskiTT force-pushed the pjanevski/chip_info branch 2 times, most recently from fe7964e to 900fda1 Compare January 29, 2025 13:35
device/tt_device/blackhole_tt_device.cpp Show resolved Hide resolved
device/api/umd/device/tt_device/blackhole_tt_device.h Outdated Show resolved Hide resolved
tests/blackhole/CMakeLists.txt Show resolved Hide resolved
@pjanevskiTT pjanevskiTT force-pushed the pjanevski/blackhole_telemetry branch from fe09713 to 3960fdb Compare February 5, 2025 13:19
@pjanevskiTT pjanevskiTT force-pushed the pjanevski/blackhole_telemetry branch 3 times, most recently from e07a0a3 to 30421f4 Compare February 6, 2025 11:57
Base automatically changed from pjanevski/blackhole_telemetry to main February 6, 2025 12:13
@pjanevskiTT pjanevskiTT enabled auto-merge (squash) February 6, 2025 12:37
@pjanevskiTT pjanevskiTT merged commit 395b1cf into main Feb 6, 2025
19 checks passed
@pjanevskiTT pjanevskiTT deleted the pjanevski/chip_info branch February 6, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants